Skip to content

[Web] Add show/hide password toggle on login pages#7155

Open
EzxD wants to merge 2 commits intomailcow:stagingfrom
EzxD:feat/show-password-toggle
Open

[Web] Add show/hide password toggle on login pages#7155
EzxD wants to merge 2 commits intomailcow:stagingfrom
EzxD:feat/show-password-toggle

Conversation

@EzxD
Copy link
Copy Markdown

@EzxD EzxD commented Mar 21, 2026

Contribution Guidelines

What does this PR include?

Short Description

Adds a show/hide password toggle button (eye icon) to the password field on all three login pages: user, admin, and domain admin.

Clicking the button toggles the field between type="password" and type="text" and switches between bi-eye and bi-eye-slash icons accordingly.

The button uses Bootstrap's input-group-text class so its border and styling are consistent with the lock icon on the left side of the input group.

Closes #6893

Affected Containers

  • nginx (web UI templates only — no backend changes)

Did you run tests?

What did you tested?

Tested all three login pages (user, admin, domain admin) using a standalone HTML preview with Bootstrap 5 and Bootstrap Icons.

What were the final results? (Awaited, got)

  • Awaited: eye icon appears at the right of the password field; clicking it reveals the password and switches to eye-slash icon; clicking again hides it.
  • Got: behaviour matches expectation on all three login pages. Button border is visually consistent with the rest of the input group.
CleanShot 2026-03-21 at 16 03 28

EzxD added 2 commits March 21, 2026 15:54
Adds an eye icon button to the password field on the user, admin, and
domain admin login pages. Clicking it toggles between hidden and visible
password text, using Bootstrap Icons (bi-eye / bi-eye-slash).

Closes mailcow#6893
Replaces btn-outline-secondary with input-group-text so the toggle
button border matches the surrounding input group consistently.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant